docs: add compiler-smoke-e2e to architecture tree and ado-script reference#1651
Draft
github-actions[bot] wants to merge 1 commit into
Draft
docs: add compiler-smoke-e2e to architecture tree and ado-script reference#1651github-actions[bot] wants to merge 1 commit into
github-actions[bot] wants to merge 1 commit into
Conversation
…rence The scripts/ado-script/src/compiler-smoke-e2e/ directory was not listed in the AGENTS.md architecture tree or the docs/ado-script.md directory listing, even though executor-e2e and trigger-e2e (the other non-bundle test harnesses) were both documented. - Add compiler-smoke-e2e to the scripts/ado-script/src/ tree in AGENTS.md - Add compiler-smoke-e2e entry to the directory listing in docs/ado-script.md - Add a 'Test-only, not shipped' callout in docs/ado-script.md matching the existing callouts for executor-e2e and trigger-e2e Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Agent Documentation Update
Findings
scripts/ado-script/src/compiler-smoke-e2e/exists as a real source directory but was absent from the architecture tree, while the two other non-bundle test harnesses (executor-e2e,trigger-e2e) were both listedAGENTS.md,docs/ado-script.mdHow found: Cross-checked every directory under
scripts/ado-script/src/against the architecture tree inAGENTS.mdand the directory listing indocs/ado-script.md. Theexecutor-e2eandtrigger-e2eharnesses were documented (both as tree entries and as "Test-only, not shipped" callouts), butcompiler-smoke-e2e— the third non-bundle harness — was missing from both.The testing section of
AGENTS.mdalready mentionedtests/compiler-smoke-e2e/(the fixture files), confirming the feature is known, but the TypeScript orchestrator inscripts/ado-script/src/compiler-smoke-e2e/was not in the source tree listing.Applied Fixes
compiler-smoke-e2e/entry to thescripts/ado-script/src/tree inAGENTS.md, positioned betweenexecutor-e2eandprepare-pr-base(consistent with the actual directory order)compiler-smoke-e2e/entry to the directory listing indocs/ado-script.mdcompiler-smoke-e2e" callout indocs/ado-script.mdadjacent to the existingexecutor-e2eandtrigger-e2ecallouts, describing what it does and confirming it is not part of the runtime bundleNotes
All other audited areas were accurate:
docs/*.mdpages are reachable from the AGENTS.md index; all index links resolvedocs/engines.mdexists as a redirect page (not needing an index entry)claude-opus-4.7) matchesDEFAULT_COPILOT_MODELinsrc/engine.rsand is consistent acrossdocs/engine.md,prompts/, and the testssrc/safe_outputs/are documented indocs/safe-outputs.mdsrc/compile/codemods/are documented indocs/codemods.mdFrontMatterstruct insrc/compile/types.rsvariable-groups:limitation (not valid forjob/stagetargets) is correctly documentedCreated by the agent-documentation maintainer workflow.